Skip to content

feat: add a utility to collate readmes from multiple versions.#329

Open
sayakpaul wants to merge 2 commits intomainfrom
collate-readme-from-version
Open

feat: add a utility to collate readmes from multiple versions.#329
sayakpaul wants to merge 2 commits intomainfrom
collate-readme-from-version

Conversation

@sayakpaul
Copy link
Member

Fixes #315

I am guessing we should incorporate this utility within the builder (perhaps the build and upload command) or something?

@sayakpaul sayakpaul requested review from danieldk and drbh March 11, 2026 03:09
@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@@ -0,0 +1,17 @@
### kernels collate-readme

Use `kernels collate-readme <repo_id>` to generate a README for the `main`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Repo READMEs are instantiated cards. Shouldn't the versions be enumerated by card filling?

Copy link
Member Author

@sayakpaul sayakpaul Mar 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry didn't get it. Did you mean #287 should have a section on "Versions" in the card template and populate it automatically?

Even if that's the case, I think the PR could still have its own scope because we're tackling the README that will be included in the main revision of the kernel repo. The builder will automatically generate the cards and push to particular revisions in which case the README will still likely stay blank. Or am I missing something?

Copy link
Member

@danieldk danieldk Mar 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For now I think main will be the default branch that is shown? Then main's README should be the instantiated card of the most recent version. I am not sure if all the version branches need individual cards, since we do not want people to actually use older versions. We leave them around to avoid breakage for people who haven't updated their code to use the latest kernel version yet, other than that, we should hide the old versions as much as possible.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think it makes sense ^

@sayakpaul
Copy link
Member Author

sayakpaul commented Mar 11, 2026

Ah this makes sense. Let me push some changes and get it ready! I think we should make the version policies a bit more visible on our docs. Potentially, also warn users if newer versions are available and if they using older ones?

@danieldk
Copy link
Member

Potentially, also warn users if newer versions are available and if they using older ones?

Yeah, a warn-level log would be awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

README utility for main branches on the build repos on Hub

4 participants